Skip to content

Plugin to display lint errors on project build - #2244

Open
baptistecs wants to merge 3 commits into
nicklockwood:developfrom
baptistecs:swiftformat-linter-build-tool-plugin
Open

Plugin to display lint errors on project build#2244
baptistecs wants to merge 3 commits into
nicklockwood:developfrom
baptistecs:swiftformat-linter-build-tool-plugin

Conversation

@baptistecs

@baptistecs baptistecs commented Oct 10, 2025

Copy link
Copy Markdown

This method allows to:

  • Use SwiftFormat as a linter, to prevent project developers to build non formatted code.
  • Keep the ENABLE_USER_SCRIPT_SANDBOXING (aka "User Script Sandboxing") option set to YES.
  • Make sure that all project developers use the same SwiftFormat version and config file.

@nicklockwood
nicklockwood force-pushed the develop branch 5 times, most recently from 1361777 to 90e8e82 Compare October 17, 2025 20:05
@baphony

baphony commented Oct 20, 2025

Copy link
Copy Markdown

Hello,

No rush, I am just wondering if there is something that I missed for this PR to be reviewed, or if I just need to wait for someone to have the time to look at it?

@calda @Blackjacx @mannylopez

Best regards,
Baptiste

@nicklockwood

nicklockwood commented Oct 27, 2025

Copy link
Copy Markdown
Owner

@baphony apologies for the delay, I will try to get to this soon. I have no objection to this in principle, but I don't use compiler plugins myself so I will need to familiarize myself with them in order to properly review the PR.

@nicklockwood
nicklockwood force-pushed the develop branch 10 times, most recently from c71af1a to 2a9a8dd Compare November 17, 2025 07:25
@baphony

baphony commented Nov 28, 2025

Copy link
Copy Markdown

@baphony apologies for the delay, I will try to get to this soon. I have no objection to this in principle, but I don't use compiler plugins myself so I will need to familiarize myself with them in order to properly review the PR.

@nicklockwood No worries, I have added some details in the REAME file, but please let me know if you have any question, I will try to help

@nicklockwood
nicklockwood force-pushed the develop branch 8 times, most recently from fc88aeb to 9348f0d Compare December 5, 2025 07:46
@baptistecs

baptistecs commented Jan 28, 2026

Copy link
Copy Markdown
Author

Hi @nicklockwood

Would it be possible to merge this?

@calda
calda force-pushed the develop branch 2 times, most recently from 7639e05 to babf646 Compare February 21, 2026 17:27
@calda
calda force-pushed the develop branch 6 times, most recently from 135a9bc to cf424cb Compare April 27, 2026 14:19
@calda
calda force-pushed the develop branch 3 times, most recently from fe8affb to 02e65e0 Compare July 6, 2026 16:14
@nicklockwood
nicklockwood force-pushed the develop branch 3 times, most recently from 5908303 to 741c3e4 Compare September 16, 2026 06:51
@nicklockwood
nicklockwood force-pushed the develop branch 3 times, most recently from e668206 to e67a08f Compare September 30, 2026 08:20
@nicklockwood
nicklockwood force-pushed the develop branch 4 times, most recently from 0a32615 to 9d33ebe Compare October 16, 2026 23:21

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@baptistecs @baphony @nicklockwood